Skip to content

add Embedded annotations#61

Merged
karle0wne merged 9 commits into
masterfrom
embeded-annotations
May 15, 2026
Merged

add Embedded annotations#61
karle0wne merged 9 commits into
masterfrom
embeded-annotations

Conversation

@karle0wne
Copy link
Copy Markdown
Contributor

@karle0wne karle0wne commented May 8, 2026

по изменениям в старом коде - убрал try-with-resources при создании контейнеров а то он по сути закрывается до старта (либо флап) . и второе - было
} else {
cleanup/deleteTopics(...);
}
и такой режим не учитывает в кафке,\постргре режим аннотации синглтон или нет , ща учитывает

по новыым эмдедеам

  • EmbeddedKafkaTestExtension#recordsToDelete очищает данные топиков перед каждым тест методом (вычиляет последний offset и и удаляет до туда все записи) .
  • кафка и постргря нет режима синглтона потмоу что они легковесные и по цене оптимально что мы на каждый тест класс их заново поднимаем (все таки не докер)

@karle0wne karle0wne requested a review from a team as a code owner May 8, 2026 07:51
ggmaleva
ggmaleva previously approved these changes May 15, 2026
@karle0wne karle0wne merged commit 4ca557d into master May 15, 2026
6 checks passed
@karle0wne karle0wne deleted the embeded-annotations branch May 15, 2026 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants